-
Notifications
You must be signed in to change notification settings - Fork 2.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[MNG-7417] Several classes do not set properties properly for building requests #306
Conversation
...ider/src/main/java/org/apache/maven/repository/internal/DefaultArtifactDescriptorReader.java
Outdated
Show resolved
Hide resolved
b916126
to
80f6adb
Compare
@slachiewicz This is the actual problem: https://issues.apache.org/jira/browse/MNG-7417 @slachiewicz If you don't mind I will pick this one up. |
...ider/src/main/java/org/apache/maven/repository/internal/DefaultArtifactDescriptorReader.java
Outdated
Show resolved
Hide resolved
…g requests This closes apache#306
maven-compat/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java
Outdated
Show resolved
Hide resolved
…g requests This closes apache#306
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@slachiewicz I have reverted the block in Maven Compat because we should touch this module as little as possible. The rest is just fine.
If I don't hear any objections I will merge this into master and 3.8.x tomorrow.
I think we should pass also user properties when building a Model for a pom.
Locally tests passed for https://issues.apache.org/jira/browse/MNG-5982
Is this a good way to fix the problem? How ITs should look?